Skip to content

Issue #244 Initial draft of performance testing page including summarized case study of DSPT use of APDEX - #245

Merged
andyblundell merged 31 commits into
mainfrom
Apdex
May 3, 2022
Merged

andyblundell merged 31 commits into
mainfrom
Apdex

Conversation

@chrisclarknhsnet

Copy link
Copy Markdown
Contributor

As per discussions with Andy

andyblundell and others added 2 commits February 14, 2022 12:12
Using APDEX for performance testing
Issue #244

Initial draft of performance testing page including summarized case study of DSPT use of APDEX
@chrisclarknhsnet
chrisclarknhsnet requested a review from a team as a code owner February 16, 2022 17:12

@andyblundell andyblundell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great - will re-review tomorrow, in the meantime have suggested one extra section. Is there any code from the case study that would be worth sharing? Maybe the Jenkins config snippet? Obviously your jmeter config is bespoke.

Comment thread practices/performancetesting.md Outdated

@stefaniuk stefaniuk left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be ok to change the location of the new images and the naming convention for the files, please? E.g. practices/images/dspt-case-study-aggregated-apdex-scores.png

Comment thread practices/performancetesting.md
Comment thread practices/performancetesting.md
@stefaniuk

Copy link
Copy Markdown
Contributor

This case study reads really well. Nice work!

Extra link from testing to performance-testing
Extra link from performance-testing to testing
Comment thread practices/performancetesting.md Outdated
@chrisclarknhsnet

chrisclarknhsnet commented Feb 17, 2022 via email

Copy link
Copy Markdown
Contributor Author

@chrisclarknhsnet

chrisclarknhsnet commented Feb 17, 2022 via email

Copy link
Copy Markdown
Contributor Author

chrisclarknhsnet and others added 4 commits February 17, 2022 17:55
All files renamed to use the - separator for words.

New images folder added under Practices directory for general images

New case studies folder under added under practices for holding images and artifacts relating to case studies
Redo of changes due to file being renamed prior to pulling from origin causing their to be 2 copies of the file i.e. performancetesting.md and performance-testing.md.

Removed redundant performancetesting.md file
Comment thread practices/performance-testing.md
@andyblundell
andyblundell dismissed stefaniuk’s stale review May 3, 2022 16:23

images have been moved

@andyblundell
andyblundell merged commit 9cb3776 into main May 3, 2022
@andyblundell
andyblundell deleted the Apdex branch May 3, 2022 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants